Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add release id to outputs #60

Conversation

loicalbertin
Copy link
Contributor

This allows to retrieve the release ID as an output for use in next steps of a workflow

@gmazzap
Copy link

gmazzap commented Apr 15, 2021

Hi here, I just needed exactly this today.

Is there anything I can do to help in move this forward?

@loicalbertin
Copy link
Contributor Author

I can have a look on resolving conflicts if there is any chance to see this merged.
WDYT @softprops?

@Frederick888
Copy link

Much appreciated for this PR. I've got a Draft Release -> Parallel Build & Upload -> Publish Release workflow and I need this ID for https://github.com/eregon/publish-release (the built-in files doesn't work for me since I'd like to see artifacts even not all builds are successful).

@softprops Any chance we can move forward with this? Thanks.

@softprops
Copy link
Owner

Sorry for the delay. it seems like some merge conflicts still exist on this one. I’d be happy to merge after those get addressed

@loicalbertin
Copy link
Contributor Author

Hi @softprops,

No problem. Merge conflicts are solved now.

@softprops softprops merged commit cebd052 into softprops:master May 1, 2021
@loicalbertin loicalbertin deleted the feature/GH-9_Add_release_ID_to_outputs branch May 2, 2021 14:58
@loicalbertin
Copy link
Contributor Author

Hi @softprops,

Thanks for merging this PR! 👍

softprops pushed a commit that referenced this pull request Jul 25, 2021
* Add release ID to outputs

* Update README
softprops pushed a commit that referenced this pull request Jul 26, 2021
* Add release ID to outputs

* Update README
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants